Skip to content

Conversation

@adalpari
Copy link
Contributor

Description

When editing a comment in the Reader, the comment text was displaying raw HTML markup tags (e.g., <p>, <br>) instead of plain text. This made the editing experience confusing as users would see the underlying HTML structure rather than the formatted content.

Testing instructions

  1. Open the app and navigate to the Reader
  2. Find or create a post to comment.
  3. Comment it
  4. Tap on the comment to view details
  5. Tap the three dots icon to edit the comment
  • Verify: The comment text should display as plain text without any HTML tags like <p>,<br>, etc
    6. Edit the comment text and save
  • Verify: The comment is saved correctly and displays properly

Before/After
Screenshot 2025-11-27 at 14 53 25 / Screenshot 2025-11-27 at 14 54 03

@wpmobilebot
Copy link
Contributor

wpmobilebot commented Nov 27, 2025

App Icon📲 You can test the changes from this Pull Request in WordPress by scanning the QR code below to install the corresponding build.
App NameWordPress
FlavorJalapeno
Build TypeDebug
Versionpr22371-4607342
Commit4607342
Direct Downloadwordpress-prototype-build-pr22371-4607342.apk
Note: Google Login is not supported on these builds.

@wpmobilebot
Copy link
Contributor

wpmobilebot commented Nov 27, 2025

App Icon📲 You can test the changes from this Pull Request in Jetpack by scanning the QR code below to install the corresponding build.
App NameJetpack
FlavorJalapeno
Build TypeDebug
Versionpr22371-4607342
Commit4607342
Direct Downloadjetpack-prototype-build-pr22371-4607342.apk
Note: Google Login is not supported on these builds.

@codecov
Copy link

codecov bot commented Nov 27, 2025

Codecov Report

❌ Patch coverage is 75.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 39.03%. Comparing base (550db72) to head (4607342).

Files with missing lines Patch % Lines
...i/comments/unified/UnifiedCommentsEditViewModel.kt 75.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##           release/26.4   #22371   +/-   ##
=============================================
  Coverage         39.03%   39.03%           
=============================================
  Files              2203     2203           
  Lines            106316   106318    +2     
  Branches          15057    15057           
=============================================
+ Hits              41499    41501    +2     
  Misses            61328    61328           
  Partials           3489     3489           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@adalpari adalpari requested a review from jkmassel November 27, 2025 14:29
@adalpari adalpari changed the base branch from trunk to release/26.4 December 1, 2025 09:16
@adalpari adalpari changed the base branch from release/26.4 to trunk December 1, 2025 09:16
@adalpari adalpari force-pushed the fix/CMM-954-reader-When-editing-a-comment-markup-tags-are-shown branch from 263205c to 32c1f70 Compare December 1, 2025 09:27
@adalpari adalpari changed the base branch from trunk to release_notes/26.4 December 1, 2025 09:27
@adalpari adalpari changed the base branch from release_notes/26.4 to release/26.4 December 1, 2025 09:31
@dangermattic
Copy link
Collaborator

1 Warning
⚠️ PR is not assigned to a milestone.

Generated by 🚫 Danger

@adalpari
Copy link
Contributor Author

adalpari commented Dec 1, 2025

Retargeting to: release/26.4

@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 1, 2025

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants